From: Olaf Hering Date: Fri, 10 Oct 2014 08:32:24 +0000 (+0200) Subject: Use C xenstored with --disable-ocamltools X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~4193 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/success//%22http:/www.example.com/cgi/success/?a=commitdiff_plain;h=d45a0826d00d49d539b89af5e9e8edde6e8907e8;p=xen.git Use C xenstored with --disable-ocamltools When xen is configured with --disable-ocamltools then oxenstored will still be the default if all required ocaml devel packages are installed. The reason is that xenstored.m4 is included before the ocamltools check. And xenstored.m4 does not use the result of --disable-ocamltools. Rearrange the code to check first for the presence of ocaml, then check --disable-ocamltools and finally decide what sort of xenstored should be the default. Signed-off-by: Olaf Hering Cc: Ian Campbell Cc: Ian Jackson Cc: Stefano Stabellini Cc: Wei Liu Acked-by: Ian Campbell [ ijc -- ran autogen.sh as requested] --- diff --git a/m4/xenstored.m4 b/m4/xenstored.m4 index 9c31a7f899..fb77b4a514 100644 --- a/m4/xenstored.m4 +++ b/m4/xenstored.m4 @@ -1,5 +1,5 @@ AC_DEFUN([AX_XEN_OCAML_XENSTORE_CHECK], [ - AS_IF([test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"], [ + AS_IF([test "$ocamltools" = "n"], [ AC_MSG_ERROR([Missing ocaml dependencies for oxenstored, try installing ocaml ocaml-compiler-libs ocaml-runtime ocaml-findlib]) ]) ]) @@ -7,7 +7,7 @@ AC_DEFUN([AX_XEN_OCAML_XENSTORE_CHECK], [ AC_DEFUN([AX_XEN_OCAML_XENSTORE_DEFAULTS], [ xenstore="oxenstored" xenstored=$sbindir/oxenstored - AS_IF([test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"], [ + AS_IF([test "$ocamltools" = "n"], [ xenstore="xenstored" xenstored=$sbindir/xenstored ]) diff --git a/tools/configure b/tools/configure index 491864f1a0..e44e754177 100755 --- a/tools/configure +++ b/tools/configure @@ -656,8 +656,8 @@ CPP pyconfig PYTHONPATH CHECKPOLICY -AWK XENSTORED +AWK OCAMLFIND OCAMLBUILD OCAMLDOC @@ -3658,14 +3658,6 @@ esac - - - - - - - - # systemd.m4 - Macros to check for and enable systemd -*- Autoconf -*- # # Copyright (C) 2014 Luis R. Rodriguez @@ -6138,67 +6130,6 @@ fi - -if test "x$XENSTORE" = "x"; then : - - -# Check whether --with-xenstored was given. -if test "${with_xenstored+set}" = set; then : - withval=$with_xenstored; - if test "x$withval" = "xxenstored"; then : - - xenstore=$withval - xenstored=$sbindir/xenstored - -fi - if test "x$withval" = "xoxenstored"; then : - - xenstore=$withval - xenstored=$sbindir/oxenstored - - if test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"; then : - - as_fn_error $? "Missing ocaml dependencies for oxenstored, try installing ocaml ocaml-compiler-libs ocaml-runtime ocaml-findlib" "$LINENO" 5 - -fi - - -fi - if test "x$withval" != "xoxenstored" && test "x$withval" != "xxenstored"; then : - - as_fn_error $? "Unsupported xenstored specified, supported types: oxenstored xenstored" "$LINENO" 5 - -fi - -else - - - xenstore="oxenstored" - xenstored=$sbindir/oxenstored - if test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"; then : - - xenstore="xenstored" - xenstored=$sbindir/xenstored - -fi - - -fi - - -fi - - - XENSTORE=$xenstore - - if test "x$XENSTORED" = "x"; then : - - XENSTORED=$xenstored - -fi - - - for ac_prog in gawk mawk nawk awk do # Extract the first word of "$ac_prog", so it can be a program name with args. @@ -6298,6 +6229,76 @@ fi fi fi + + + + + + + + + + +if test "x$XENSTORE" = "x"; then : + + +# Check whether --with-xenstored was given. +if test "${with_xenstored+set}" = set; then : + withval=$with_xenstored; + if test "x$withval" = "xxenstored"; then : + + xenstore=$withval + xenstored=$sbindir/xenstored + +fi + if test "x$withval" = "xoxenstored"; then : + + xenstore=$withval + xenstored=$sbindir/oxenstored + + if test "$ocamltools" = "n"; then : + + as_fn_error $? "Missing ocaml dependencies for oxenstored, try installing ocaml ocaml-compiler-libs ocaml-runtime ocaml-findlib" "$LINENO" 5 + +fi + + +fi + if test "x$withval" != "xoxenstored" && test "x$withval" != "xxenstored"; then : + + as_fn_error $? "Unsupported xenstored specified, supported types: oxenstored xenstored" "$LINENO" 5 + +fi + +else + + + xenstore="oxenstored" + xenstored=$sbindir/oxenstored + if test "$ocamltools" = "n"; then : + + xenstore="xenstored" + xenstored=$sbindir/xenstored + +fi + + +fi + + +fi + + + XENSTORE=$xenstore + + if test "x$XENSTORED" = "x"; then : + + XENSTORED=$xenstored + +fi + + + if test "x$xsmpolicy" = "xy"; then : # check for a checkpolicy binary with support for -t xen diff --git a/tools/configure.ac b/tools/configure.ac index 98b245567f..f584798dd5 100644 --- a/tools/configure.ac +++ b/tools/configure.ac @@ -75,7 +75,6 @@ m4_include([../m4/extfs.m4]) m4_include([../m4/fetcher.m4]) m4_include([../m4/ax_compare_version.m4]) m4_include([../m4/paths.m4]) -m4_include([../m4/xenstored.m4]) m4_include([../m4/systemd.m4]) AX_XEN_EXPAND_CONFIG() @@ -268,9 +267,6 @@ AX_PATH_PROG_OR_FAIL([PERL], [perl]) AC_PROG_OCAML AC_PROG_FINDLIB -AX_XENSTORE_OPTIONS -AX_XENSTORE_SET - AS_IF([test "x$ocamltools" = "xy"], [ AS_IF([test "x$OCAMLC" = "xno" || test "x$OCAMLFIND" = "xno"], [ AS_IF([test "x$enable_ocamltools" = "xyes"], [ @@ -284,6 +280,11 @@ AS_IF([test "x$ocamltools" = "xy"], [ ]) ]) ]) + +m4_include([../m4/xenstored.m4]) +AX_XENSTORE_OPTIONS +AX_XENSTORE_SET + AS_IF([test "x$xsmpolicy" = "xy"], [ AC_PROG_CHECKPOLICY AS_IF([test "x$CHECKPOLICY" = "xno"], [